Skip to content

Conversation

@yordan-st
Copy link
Contributor

@yordan-st yordan-st commented Nov 20, 2025

Pull request type


Description

TODO

  • Rename Header to ColumnContainer
  • Create new component ColumnHeader and extract JSX from ColumnContainer
  • Rewrite getSortIcon to SortIcon
  • Create folder features/column
  • Add HeaderDragnDrop.store.ts
  • Extract hooks from GridHeader to HeaderDragnDrop store
  • Rewrite useDraggable to ColumnHeader.viewModel.ts with mobx
  • Rewrite ColumnResizer: use hooks to get ColumnStore and Column

@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch from 0f9af81 to fb41b35 Compare November 21, 2025 08:22
@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch from 5f3b6d2 to 0520f25 Compare November 21, 2025 14:32
@yordan-st yordan-st marked this pull request as ready for review November 26, 2025 08:21
@yordan-st yordan-st requested a review from a team as a code owner November 26, 2025 08:21
@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch 2 times, most recently from 8e73071 to 7bb9a06 Compare November 26, 2025 15:46
@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch 2 times, most recently from 75213b1 to bb74b16 Compare November 28, 2025 10:27
Base automatically changed from iobuhov_next to main December 2, 2025 08:15
@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch from 5f254d1 to 3831723 Compare December 2, 2025 09:19
@yordan-st yordan-st requested a review from iobuhov December 2, 2025 09:24
@yordan-st yordan-st force-pushed the feat/WC-1784_DG-coldrag-enhance branch from 2f6bbaf to eeb1665 Compare December 2, 2025 09:47
Copy link
Collaborator

@iobuhov iobuhov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants